Skip to content

Rework initialize_linear_solver#107

Merged
pjaap merged 2 commits intomasterfrom
feature/rework-init
Feb 12, 2026
Merged

Rework initialize_linear_solver#107
pjaap merged 2 commits intomasterfrom
feature/rework-init

Conversation

@pjaap
Copy link
Copy Markdown
Member

@pjaap pjaap commented Feb 6, 2026

This removes the LP property of the SolverConfiguration since all information is
stored in the linsolver.

The initializer is now triggered much later in the solve-call: after system matrix
and right hand side are known. This is important when restrictions are in use.

Moreover, we do not need to assemble a u vector for the linear solver. This is done
in the init-call.

This removes the `LP` property of the `SolverConfiguration` since all information is
stored in the `linsolver`.

The initializer is now triggered much later in the `solve`-call: after system matrix
and right hand side are known. This is important when restrictions are in use.

Moreover, we do not need to assemble a `u` vector for the linear solver. This is done
in the init-call.
@pjaap pjaap force-pushed the feature/rework-init branch from c764a0a to d0aebce Compare February 12, 2026 12:25
@pjaap pjaap changed the title WIP: remove SC.LP; init when system is known; do not assemble u Rework initialize_linear_solver Feb 12, 2026
@pjaap
Copy link
Copy Markdown
Member Author

pjaap commented Feb 12, 2026

Seems to work fine after a few adjustments.

@pjaap pjaap requested a review from chmerdon February 12, 2026 12:26
Copy link
Copy Markdown
Member

@chmerdon chmerdon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me

@pjaap pjaap merged commit 55b2d7f into master Feb 12, 2026
8 of 11 checks passed
@pjaap pjaap deleted the feature/rework-init branch February 12, 2026 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants